Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updating .travis.yml and package.json to require node 0.10 or newer #25

Merged
merged 2 commits into from
Feb 18, 2015
Merged

Conversation

jtatum
Copy link
Contributor

@jtatum jtatum commented Feb 17, 2015

The versions of grunt specified in package.json do not run on node 0.8. Grunt and its cohorts aren't required for the package itself or the test cases, so .travis.yml will remove them from package.json prior to installing dependencies and running tests.

Successful Travis build: https://travis-ci.org/jtatum/node-slack-client/builds/51145556

The versions of grunt specified in package.json do not run on node 0.8.
Grunt and its cohorts aren't required for the package itself or the test
cases, so .travis.yml will remove them from package.json prior to installing
dependencies and running tests.
@evansolomon
Copy link
Contributor

Is anyone using Node 08 these days? I think I'd actually drop 0.8 support/testing, but someone from Slack should probably make that choice.

@paulhammond
Copy link
Contributor

We're ok with dropping it!

@jtatum
Copy link
Contributor Author

jtatum commented Feb 18, 2015

@jtatum jtatum changed the title Updating .travis.yml to remove grunt dependencies Updating .travis.yml and package.json to require node 0.10 or newer Feb 18, 2015
evansolomon added a commit that referenced this pull request Feb 18, 2015
Updating .travis.yml and package.json to require node 0.10 or newer
@evansolomon evansolomon merged commit cc157c5 into slackapi:master Feb 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants